Skip to content

Remove executable Dolt validator package - #9

Merged
TonisOrmisson merged 5 commits into
mainfrom
agent/remove-dolt-validator
Aug 4, 2026
Merged

Remove executable Dolt validator package#9
TonisOrmisson merged 5 commits into
mainfrom
agent/remove-dolt-validator

Conversation

@TonisOrmisson

@TonisOrmisson TonisOrmisson commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What changed

  • removes the installable openstatspec-specification Python package
  • removes the executable Dolt declaration validator, its tests, and its tool wrapper
  • removes package installation and validator-test steps from specification CI
  • removes the package-specific PyPI publishing workflow
  • removes the unreleased 0.2.1 package path from the changelog and roadmap
  • removes the validator-coupled Dolt declaration manifest and empty evidence-registry guidance
  • records that Transformation Plan/frontend profile 0.2 is already merged

What remains

  • the authoritative Dolt dialect profile in sql/dialect-profile-baseline.json
  • existing cross-adapter capability declaration requirements in the normative SQL profile documents
  • existing specification repository validation
  • all pending adapter, service-matrix, exact-commit, and tag-protection gates for v0.3.0

Architecture boundary

The specification repository publishes normative documentation, schemas, and declarative fixtures. Executable adapter/declaration validation belongs in implementation or adapter repositories.

Validation

Release impact

No corrective release is required. openstatspec-specification==0.2.1 was never published, and no v0.2.1 tag or GitHub Release exists. Keep this work under Unreleased; the next possible specification release remains v0.3.0 after the documented gates pass.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cba97649d7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread sql/dolt-adapter-declaration-schema.json Outdated

Copy link
Copy Markdown
Contributor Author

Resolved the Codex P2 on exact head 99dd1cee378be918cbcd066b9fd934eb5a92aa8c without restoring executable validation. The remaining declaration manifest and empty declaration/evidence registry guidance were removed entirely because their cross-artifact evidence invariants cannot be expressed by that metadata file alone. The authoritative Dolt dialect profile and existing normative capability-declaration requirements remain unchanged; concrete declarations and evidence belong in implementation/adapter repositories with their own validation. Exact-head push and PR CI are green.

Copy link
Copy Markdown
Contributor Author

@codex review

Please review exact head 99dd1cee378be918cbcd066b9fd934eb5a92aa8c. The executable/package validator stack and its declaration/evidence registry artifacts are removed; the authoritative Dolt dialect profile remains unchanged. Please assess the specification/implementation boundary and release metadata.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: 99dd1cee37

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@TonisOrmisson
TonisOrmisson merged commit 6907f96 into main Aug 4, 2026
2 checks passed
@TonisOrmisson
TonisOrmisson deleted the agent/remove-dolt-validator branch August 4, 2026 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant